Skip to content

Analyzed various character ex-field; Added Player::TrailImage - #29

Open
jyanf wants to merge 20 commits into
SokuDev:devfrom
jyanf:only4PR
Open

Analyzed various character ex-field; Added Player::TrailImage#29
jyanf wants to merge 20 commits into
SokuDev:devfrom
jyanf:only4PR

Conversation

@jyanf

@jyanf jyanf commented Sep 1, 2026

Copy link
Copy Markdown

No description provided.

adjusted tenshi's field signedness
suwako and iku fields supplement
some unsure/unused fields are retyped by checking init vfunc
renamed suwako 0x891 bool
@jyanf jyanf changed the title Half-analyzed Alice ex-fields; Added Player::TrailImage Analyzed various character ex-field; Added Player::TrailImage Sep 2, 2026
Comment thread src/Player.hpp Outdated

float orreriesRotatePhase;// -=speed
float orreriesRotateSpeed;//type0: 3, type1: 12, type8: 0.5, type50: 12
float orreriesThrowTargetX, orreriesThrowTargetY;//record decided target pos when throwing out

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be a Vector2f

Comment thread src/Player.cpp Outdated
reinterpret_cast<void (__fastcall*)(void*)>(0x433a50)(&this->unknownObject);
// clone buffer clear
//reinterpret_cast<void (__fastcall*)(void*)>(0x433a50)(&this->cloneBuffer);
this->cloneBuffer.clear();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

~Deque already clears so I don't think it's required. Keeping the old code also is redundant with the git.

Comment thread src/Player.cpp Outdated
Player(info)
{
memset(this->unknown8C0, 0, 0x10); // [0x8c0-0x8d0] = 0
//memset(&this->cloneBuffer.m_table, 0, 0x10);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove unused code

Comment thread src/Player.hpp
@PinkySmile

Copy link
Copy Markdown
Member

I'll test it against CharacterEngine and see if anything breaks before merging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants